-
Notifications
You must be signed in to change notification settings - Fork 17
loosen rich dep and update example render #142
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
loosen rich dep and update example render #142
Conversation
alpha published and smoke tested: |
what if we just pin the rick dependency to 14 and update the examples to remove the conditionals? @filip-michalsky |
This is unfortunately not possible as this PR is fighting the dependency hell between python stagehand and CrewAI tools. The point of this PR is to allow update of dependencies between CrewAI tools and our updated stagehand package. See here: it does not build.
Hence we need rich max of 13.7.0 unless we update I can update the example to not have the conditional there - that is just a rendering thing. |
got it, I think we can move down to 13.7 instead? It seems simpler than having to do conditional logic for formatting |
yep, this has been done. Thank you for the review. Just need the green whenever you get a chance, ty |
why
Need to update deps to merge new pypi stagehand dep to CrewAI
what changed
Loosened rich - this is a non-core dep for example rendering, not in core lib
test plan
CI